Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Quick fix for the updated website theaming to access the correct css files #7031

Merged
merged 1 commit into from
Apr 5, 2021

Conversation

xFrednet
Copy link
Member

@xFrednet xFrednet commented Apr 4, 2021

This fixes a problem from #7030 that the service used to access css files was blocked by GitHub pages due to SSL_ERROR_BAD_CERT_DOMAIN. The css file loading worked fine during local development. The browser probably disabled some security options due to the local address.

This fix works locally and should also work online as it references the direct css files used by the mdBook User Guide the disadvantage of this is that refactorings within the mdBook project can have effects on the theme loading of Clippy. This PR is therefor more meant as a quick fix until I find a better solution.

I've tested these changes using the page editor in the browser and can now confirm that they work :)

changelog: none

r? @llogiq

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Apr 4, 2021
@xFrednet xFrednet force-pushed the 5234-quick-fix-dark-mode-themes branch from e05d213 to da04e87 Compare April 4, 2021 18:15
@llogiq
Copy link
Contributor

llogiq commented Apr 5, 2021

Thanks for the quick fix. @bors r+

@bors
Copy link
Collaborator

bors commented Apr 5, 2021

📌 Commit da04e87 has been approved by llogiq

@bors
Copy link
Collaborator

bors commented Apr 5, 2021

⌛ Testing commit da04e87 with merge e315437...

@bors
Copy link
Collaborator

bors commented Apr 5, 2021

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: llogiq
Pushing e315437 to master...

@bors bors merged commit e315437 into rust-lang:master Apr 5, 2021
@xFrednet xFrednet deleted the 5234-quick-fix-dark-mode-themes branch July 28, 2021 16:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants